Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ECAL and GEM tags to 122X GTs #36083

Merged
merged 1 commit into from
Nov 11, 2021

Conversation

malbouis
Copy link
Contributor

@malbouis malbouis commented Nov 10, 2021

PR description:

This PR is to include the ECAL Linearization Constant tag in the mcRun4 and mcRun3 GTs, as requested in https://hypernews.cern.ch/HyperNews/CMS/get/calibrations/4529.html

It also includes a new EcalLaserAPDPNRatios ECAL tag in the Run3 offline GT.

There is also the inclusion of the GEMeMap tag in the run3 offline GT, as requested in https://hypernews.cern.ch/HyperNews/CMS/get/calibrations/4524.html.

The tags included in the GTs are:
ECAL Linearization Constant (in mcRun4 and mcRun3 GTs except the design one), EcalTPGLinearizationConst_UL_2018_mc_EE_BTCP_116_SIC_1

EcalLaserAPDPNRatios (in Run3 offline GT), EcalLaserAPDPNRatios_offline_Run3_v1

GEMeMap (in the Run3 offline GT), GEMeMap_Full_v2

Since all 122X Queues have been created we also updated all other GTs in autoCond to 122X.

The diffs in GTs are below. We expect to see differences only in the mcRun4 and mcRun3 (except the design one) and Run3 offline GTs:

run1_design
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_mcRun1_design_v1/121X_mcRun1_design_v6

run1_mc
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_mcRun1_realistic_v1/121X_mcRun1_realistic_v6

run1_mc_hi
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_mcRun1_HeavyIon_v1/121X_mcRun1_HeavyIon_v7

run2_mc_50ns
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_mcRun2_startup_v1/121X_mcRun2_startup_v8

run2_mc_l1stage1
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_mcRun2_asymptotic_l1stage1_v1/121X_mcRun2_asymptotic_l1stage1_v8

run2_design
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_mcRun2_design_v1/121X_mcRun2_design_v8

run2_mc_pre_vfp
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_mcRun2_asymptotic_preVFP_v1/121X_mcRun2_asymptotic_preVFP_v8

run2_mc
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_mcRun2_asymptotic_v1/121X_mcRun2_asymptotic_v8

run2_mc_cosmics
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_mcRun2cosmics_asymptotic_deco_v1/121X_mcRun2cosmics_asymptotic_deco_v8

run2_mc_hi
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_mcRun2_HeavyIon_v1/121X_mcRun2_HeavyIon_v8

run2_mc_pa
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_mcRun2_pA_v1/121X_mcRun2_pA_v8

run2_data
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_dataRun2_v1/121X_dataRun2_v11

run2_data_HEfail
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_dataRun2_HEfail_v1/121X_dataRun2_HEfail_v11

run2_data_relval
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_dataRun2_relval_v1/121X_dataRun2_relval_v11

run2_data_promptlike_hi
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_dataRun2_PromptLike_HI_v1/121X_dataRun2_PromptLike_HI_v10

run3_hlt
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_dataRun3_HLT_v1/121X_dataRun3_HLT_v11

run2_hlt_relval
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_dataRun2_HLT_relval_v1/121X_dataRun2_HLT_relval_v11

run3_data_express
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_dataRun3_Express_v1/121X_dataRun3_Express_v11

run3_data_prompt
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_dataRun3_Prompt_v1/121X_dataRun3_Prompt_v10

run3_data
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_dataRun3_v1/121X_dataRun3_v10

phase1_2017_design
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_mc2017_design_v1/121X_mc2017_design_v9

phase1_2017_realistic
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_mc2017_realistic_v1/121X_mc2017_realistic_v9

phase1_2017_cosmics
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_mc2017cosmics_realistic_deco_v1/121X_mc2017cosmics_realistic_deco_v9

phase1_2017_cosmics_peak
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_mc2017cosmics_realistic_peak_v1/121X_mc2017cosmics_realistic_peak_v9

phase1_2018_design
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_upgrade2018_design_v1/121X_upgrade2018_design_v8

phase1_2018_realistic
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_upgrade2018_realistic_v1/121X_upgrade2018_realistic_v8

phase1_2018_realistic_rd
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_upgrade2018_realistic_RD_v1/121X_upgrade2018_realistic_RD_v8

phase1_2018_realistic_hi
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_upgrade2018_realistic_HI_v1/121X_upgrade2018_realistic_HI_v8

phase1_2018_realistic_HEfail
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_upgrade2018_realistic_HEfail_v1/121X_upgrade2018_realistic_HEfail_v8

phase1_2018_cosmics
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_upgrade2018cosmics_realistic_deco_v1/121X_upgrade2018cosmics_realistic_deco_v10

phase1_2018_cosmics_peak
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_upgrade2018cosmics_realistic_peak_v1/121X_upgrade2018cosmics_realistic_peak_v8

phase1_2021_design
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_mcRun3_2021_design_v1/121X_mcRun3_2021_design_v16

phase1_2021_realistic
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_mcRun3_2021_realistic_v1/121X_mcRun3_2021_realistic_v18

phase1_2021_cosmics
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_mcRun3_2021cosmics_realistic_deco_v1/121X_mcRun3_2021cosmics_realistic_deco_v18

phase1_2021_realistic_hi
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_mcRun3_2021_realistic_HI_v1/121X_mcRun3_2021_realistic_HI_v18

phase1_2023_realistic
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_mcRun3_2023_realistic_v1/121X_mcRun3_2023_realistic_v17

phase1_2024_realistic
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_mcRun3_2024_realistic_v1/121X_mcRun3_2024_realistic_v17

phase2_realistic
https://cms-conddb.cern.ch/cmsDbBrowser/diff/Prod/gts//122X_mcRun4_realistic_v1/121X_mcRun4_realistic_v8

PR validation:

runTheMatrix.py -l limited,136.897,7.23,12834.0 --ibeos -j16

if this PR is a backport please specify the original PR and why you need to backport that PR:

it is not a backport

@malbouis
Copy link
Contributor Author

test parameters:

  • workflows=136.897,7.23,12834.0

@cmsbuild
Copy link
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-36083/26558

  • This PR adds an extra 12KB to repository

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @malbouis for master.

It involves the following packages:

  • Configuration/AlCa (alca)

@cmsbuild, @malbouis, @tvami, @yuanchao, @francescobrivio can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @missirol, @mmusich, @fabiocos, @tocheng this is something you requested to watch as well.
@perrotta, @dpiparo, @qliphy you are the release manager for this.

cms-bot commands are listed here

@malbouis
Copy link
Contributor Author

please test

@cmsbuild
Copy link
Contributor

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-ec37d9/20446/summary.html
COMMIT: 458ad6e
CMSSW: CMSSW_12_2_X_2021-11-10-1100/slc7_amd64_gcc900
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/36083/20446/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

@slava77 comparisons for the following workflows were not done due to missing matrix map:

  • /data/cmsbld/jenkins/workspace/compare-root-files-short-matrix/data/PR-ec37d9/136.897_RunCosmics2021CRUZET+RunCosmics2021CRUZET+RECOCOSDRUN3+ALCACOSDRUN3+HARVESTDCRUN3
  • /data/cmsbld/jenkins/workspace/compare-root-files-short-matrix/data/PR-ec37d9/7.23_Cosmics_UP21+Cosmics_UP21+DIGICOS_UP21+RECOCOS_UP21+ALCACOS_UP21+HARVESTCOS_UP21

Summary:

  • No significant changes to the logs found
  • Reco comparison results: 6246 differences found in the comparisons
  • DQMHistoTests: Total files compared: 42
  • DQMHistoTests: Total histograms compared: 2901890
  • DQMHistoTests: Total failures: 14508
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 2887360
  • DQMHistoTests: Total skipped: 22
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 0.0 KiB( 41 files compared)
  • Checked 177 log files, 37 edm output root files, 42 DQM output files
  • TriggerResults: found differences in 5 / 41 workflows

@francescobrivio
Copy link
Contributor

GT diffs look good and as expected differences in https://cmssdt.cern.ch/SDT/jenkins-artifacts/baseLineComparisons/CMSSW_12_2_X_2021-11-10-1100+ec37d9/46786/validateJR.html appear only in mcRun3 and mcRun4 workflows.

@camendola @simonepigazzini could you confirm the differences are as expected (I could not find Marco Cipriani's git handle, so feel free to tag his as well)?

@cippy
Copy link
Contributor

cippy commented Nov 11, 2021

I had a look and the differences look as expected

@tvami
Copy link
Contributor

tvami commented Nov 11, 2021

+alca

  • GT changes according to the description
  • differences in tests are as expected + experts confirm them too!

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2)

@perrotta
Copy link
Contributor

+1

@cmsbuild cmsbuild merged commit 1066020 into cms-sw:master Nov 11, 2021
jhonatanamado added a commit to jhonatanamado/T0 that referenced this pull request Jan 17, 2022
Change GTs in referece to  cms-sw/cmssw#36083
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants